adding support for static locations #227
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi,
I recently wanted to chain two Modlishka instances together to form an authentic ruse but there was a problem that resulted in a functional patch. I've tested it on an active engagement without issue and thought it would be beneficial to contribute.
The Problem
The Solution
staticLocations
, was added that allows configuration of allowed FQDNs in theLocation
header that will not be converted to the current phish domain viaruntime.RealURLtoPhish
runtime.ReplaceStrings
Location
header should be updated viaruntime.RealURLtoPhish
The Outcome
I refer to each Modlishka server as a stage to make this easier to digest. The sections below describe each server configuration and the final request flow.
Server Configurations
This configuration allowed me to chain two Modlishka servers:
Note: Alias S1M reads like "Stage 1 Modlishka"
S1M Configuration File
S2M Configuration File
Final Request Flow
www.sharfile.com
login.microsoftonline.com
tologin.ophish365.com
login.ophish365.com
is astaticLocation
value, redirects to S2Mlogin.ophish365.com
/kmsi
resource, provoking termination and redirect back towww.sharfile.com